Skip to content

T3 Merging Tests#230

Draft
GNiendorf wants to merge 1 commit intomasterfrom
t3_merg
Draft

T3 Merging Tests#230
GNiendorf wants to merge 1 commit intomasterfrom
t3_merg

Conversation

@GNiendorf
Copy link
Member

Draft PR to test T3 merging stuff, still unclear if this will get merged.

@github-actions
Copy link

There was a problem while building and running in standalone mode. The logs can be found here.

@GNiendorf
Copy link
Member Author

/run standalone

@GNiendorf
Copy link
Member Author

/run cmssw
/run checks

@GNiendorf
Copy link
Member Author

Looks like cuts are still too loose at high pT and in some regions, I will take a look.

@slava77
Copy link

slava77 commented Jan 30, 2026

Here is a timing comparison: Reset: 188.2 -> 428.9

If it's from this PR, is the timer out of place now?

@GNiendorf
Copy link
Member Author

/run all

@SegmentLinking SegmentLinking deleted a comment from github-actions bot Jan 30, 2026
@GNiendorf
Copy link
Member Author

Here is a timing comparison: Reset: 188.2 -> 428.9

If it's from this PR, is the timer out of place now?

Maybe because it is multi-stream? Sometimes the time increases for each category can be unreliable.

@github-actions
Copy link

The PR was built and ran successfully in standalone mode. Here are some of the comparison plots.

Efficiency vs pT comparison Efficiency vs eta comparison
Fake rate vs pT comparison Fake rate vs eta comparison
Duplicate rate vs pT comparison Duplicate rate vs eta comparison

The full set of validation and comparison plots can be found here.

Here is a timing comparison:

   Evt    Hits       MD       LS      T3       T5       pLS       pT5      pT3      TC       Reset    Event     Short             Rate
   avg     30.9    378.1    272.8    125.9     52.2    697.0     11.1    127.1    132.7    188.5      1.7    2018.0    1290.0+/- 307.3     619.9   explicit[s=4] (target branch)
   avg     28.6    381.5    275.9    119.6     49.3    687.5     12.1    126.8    134.7    428.5      1.9    2246.3    1530.1+/- 370.2     682.6   explicit[s=4] (this PR)

@slava77
Copy link

slava77 commented Jan 30, 2026

Here is a timing comparison: Reset: 188.2 -> 428.9

If it's from this PR, is the timer out of place now?

Maybe because it is multi-stream? Sometimes the time increases for each category can be unreliable.

the last test came up with a similar result.
These are CPU-based, btw.

@GNiendorf
Copy link
Member Author

GNiendorf commented Jan 30, 2026

Current improvement plot, seems a decent chunk of the pT3's get extended to pT4's.

d33fc048-2f5d-4cc4-81b8-d34224892ad2

@github-actions
Copy link

The PR was built and ran successfully with CMSSW. Here are some plots.

OOTB All Tracks
Efficiency and fake rate vs pT, eta, and phi

The full set of validation and comparison plots can be found here.

@GNiendorf
Copy link
Member Author

No noticeable improvement in momentum resolution.

Screenshot 2026-01-30 at 5 11 18 PM Screenshot 2026-01-30 at 5 10 50 PM

@GNiendorf
Copy link
Member Author

Plot showing tracks are increasing length outside barrel.

Screenshot 2026-02-02 at 12 07 59 PM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants